Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #130: Грейд версий @atls/react-kratos-browser-flows #131

Closed
wants to merge 3 commits into from

Conversation

Voiislav
Copy link

No description provided.

@Voiislav Voiislav self-assigned this Oct 24, 2023
@Voiislav Voiislav added the enhancement New feature or request label Oct 24, 2023
@Voiislav Voiislav linked an issue Oct 24, 2023 that may be closed by this pull request
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обрати внимание - твой пр не прошел проверки по линту и тайпчеку.

Ошибки связаны в большинстве с разными версиями @types/react. Унифицируй версию по всему проекту либо в ручную либо сделай резолюшн в корне.

Предпочтительнее второй вариант потому что остальные пакеты давно не обновлялись и могут пойти конфликты. Поэтому лучше поставить резолюшн на 17 реакт типы, но в package.json твоего пакета поставить 18 реакт типы.

@Voiislav
Copy link
Author

@Nelfimov

но в package.json твоего пакета поставить 18 реакт типы.
C этим понятно - в package.json у kratos-browser-flows оставляю "@types/react": "18.2.31".

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

@Nelfimov
Copy link
Member

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

я обычно просто вручную прописываю резолюшн в корневом package.json. Поставь только версию последнюю в мажорной 17.

@Voiislav
Copy link
Author

Voiislav commented Oct 25, 2023

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

я обычно просто вручную прописываю резолюшн в корневом package.json. Поставь только версию последнюю в мажорной 17.

Все готово, но ПР не прошел проверку на install этапе. Прописал резолюшн только в корневом package.json и оставил 18 версию в package.json kratos-browser-flows. В резолюшнах в yarn.lock вижу 18 версию. Ошибки, судя по всему, тоже ссылаются на это, скрин ниже. Я что-то упустил?

Details

photo_2023-10-25 10 00 23

UPD: решено

@Nelfimov
Copy link
Member

  1. @emotion не выше 11.9.Х
  2. Тайпскрипт не выше 4.6.4
  3. eslint не выше 8.18
  4. После установки резолюшн надо обязательно проустановить зависимости через yarn
  5. Нейминг ветки - надо соблюдать нашу конвенцию с префиксами названий - feat, fix, chore, doc и т.д.
  6. Обрати внимание, что твой ПР затрагивает обновление всех пакетов, а в задаче - нужно грейдить только один. Поэтому во всех пакетах которые не указаны в задаче откатывай изменения.

@Voiislav
Copy link
Author

Voiislav commented Oct 25, 2023

  1. @emotion не выше 11.9.Х
  2. Тайпскрипт не выше 4.6.4
  3. eslint не выше 8.18
  4. После установки резолюшн надо обязательно проустановить зависимости через yarn
  5. Нейминг ветки - надо соблюдать нашу конвенцию с префиксами названий - feat, fix, chore, doc и т.д.
  6. Обрати внимание, что твой ПР затрагивает обновление всех пакетов, а в задаче - нужно грейдить только один. Поэтому во всех пакетах которые не указаны в задаче откатывай изменения.

Готово. Не использовал yarn upgrade-interactive, а обновил версии вручную в корневом и kratos-browser-flows package.json'ах. Проустановил зависимости через yarn. Но после yarn check возникла ошибка в другом пакете, текст ошибки:

packages/payment-widget/src/interfaces/fields.interfaces.ts
➤ YN0000: │
➤ YN0000: │ Error: Module '"react"' has no exported member 'HTMLInputTypeAttribute'.

Корневой резолюшн:

 "resolutions": {
    "@emotion/react": ">=11.9.0 <11.10.0",
    "@types/react": "17.0.2",
    "eslint": "<=8.18.0",
    "typescript": "<=4.6.4"
  },

@Nelfimov, подскажи, пожалуйста, в чем может быть причина ошибки в этом пакете? Или достаточно, чтобы yarn check не выдавал ошибки только в пакете kratos-browser-flows?

UPD: ошибок при yarn check больше нет, изменил в резолюшн "@types/react": "^17.0.0"

@Voiislav
Copy link
Author

Совершил ошибку: после изменения имени ветки создал новый ПР вместо изменения текущего.

@Nelfimov
Copy link
Member

Совершил ошибку: после изменения имени ветки создал новый ПР вместо изменения текущего.

Отменяй/сноси старый и работаем с верным.

@Voiislav Voiislav closed this Oct 25, 2023
@Voiislav Voiislav deleted the upgrade-branch branch October 25, 2023 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants